-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate cloudbuild connection to mmv1 #9537
Migrate cloudbuild connection to mmv1 #9537
Conversation
Rerun these tests in REPLAYING mode to catch issues
Tests failed due to non-determinism or randomness when the VCR replayed the response after the HTTP request was made. Please fix these to complete your PR. If you believe these test failures to be incorrect or unrelated to your change, or if you have any questions, please raise the concern with your reviewer.
|
fec23b0
to
f7b7999
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2782 insertions(+), 2138 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 13 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccCloudBuildTrigger_cloudbuildTriggerRepoExample|TestAccCloudBuildTrigger_cloudbuildTriggerPubsubWithRepoExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGleExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGheExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGithubExample|TestAccCloudbuildv2Connection_GithubConnection|TestAccCloudbuildv2Connection_GleConnection|TestAccCloudbuildv2Connection_GlePrivConnection|TestAccCloudbuildv2Connection_GlePrivUpdateConnection|TestAccCloudbuildv2Connection_GheConnection|TestAccCloudbuildv2Connection_GhePrivUpdateConnection|TestAccCloudbuildv2Connection_GleOldConnection|TestAccDataSourceGoogleServiceAccountIdToken_impersonation |
f7b7999
to
9063a6c
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2782 insertions(+), 2138 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 13 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccCloudBuildTrigger_cloudbuildTriggerRepoExample|TestAccCloudBuildTrigger_cloudbuildTriggerPubsubWithRepoExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGheExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGleExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGithubExample|TestAccCloudbuildv2Connection_GlePrivConnection|TestAccCloudbuildv2Connection_GheConnection|TestAccCloudbuildv2Connection_GithubConnection|TestAccCloudbuildv2Connection_GhePrivUpdateConnection|TestAccCloudbuildv2Connection_GlePrivUpdateConnection|TestAccCloudbuildv2Connection_GleOldConnection|TestAccCloudbuildv2Connection_GleConnection|TestAccDataSourceGoogleServiceAccountIdToken_impersonation |
Rerun these tests in REPLAYING mode to catch issues
|
7a2e6a0
to
6141e2e
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2782 insertions(+), 2138 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 14 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccCloudBuildTrigger_cloudbuildTriggerRepoExample|TestAccCloudBuildTrigger_cloudbuildTriggerPubsubWithRepoExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGithubExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGleExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGheExample|TestAccCloudbuildv2Connection_GheConnection|TestAccCloudbuildv2Connection_GithubConnection|TestAccCloudbuildv2Connection_GlePrivConnection|TestAccCloudbuildv2Connection_GleOldConnection|TestAccCloudbuildv2Connection_GlePrivUpdateConnection|TestAccCloudbuildv2Connection_GhePrivUpdateConnection|TestAccCloudbuildv2Connection_GleConnection|TestAccGKEHub2Fleet_gkehubFleetBasicExample_update|TestAccLoggingProjectSink_updatePreservesCustomWriter |
Rerun these tests in REPLAYING mode to catch issues
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2782 insertions(+), 2138 deletions(-)) |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2774 insertions(+), 2130 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 13 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccCloudBuildTrigger_cloudbuildTriggerRepoExample|TestAccCloudBuildTrigger_cloudbuildTriggerPubsubWithRepoExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGleExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGithubExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGheExample|TestAccCloudbuildv2Connection_GheConnection|TestAccCloudbuildv2Connection_GleConnection|TestAccCloudbuildv2Connection_GithubConnection|TestAccCloudbuildv2Connection_GleOldConnection|TestAccCloudbuildv2Connection_GhePrivUpdateConnection|TestAccCloudbuildv2Connection_GlePrivUpdateConnection|TestAccCloudbuildv2Connection_GlePrivConnection|TestAccLoggingProjectSink_updatePreservesCustomWriter |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2768 insertions(+), 2131 deletions(-)) |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 21 files changed, 2768 insertions(+), 2131 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccCloudBuildTrigger_cloudbuildTriggerRepoExample|TestAccCloudBuildTrigger_cloudbuildTriggerPubsubWithRepoExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGleExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGithubExample|TestAccCloudbuildv2Repository_cloudbuildv2RepositoryGheExample|TestAccCloudbuildv2Connection_GhePrivUpdateConnection|TestAccCloudbuildv2Connection_GithubConnection|TestAccCloudbuildv2Connection_GlePrivConnection|TestAccCloudbuildv2Connection_GleConnection|TestAccCloudbuildv2Connection_GlePrivUpdateConnection|TestAccCloudbuildv2Connection_GheConnection|TestAccCloudbuildv2Connection_GleOldConnection|TestAccDataprocJobIamPolicy|TestAccDataprocClusterIamPolicy|TestAccGKEHub2Fleet_gkehubFleetBasicExample_update |
Rerun these tests in REPLAYING mode to catch issues
|
Tests analyticsTotal tests: Click here to see the affected service packagesall service packages are affected Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccDataprocJobIamPolicy |
Rerun these tests in REPLAYING mode to catch issues
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we've lost several DSFs inside of objects. Does our breaking change checker not catch that? We had a way to detect these things I thought?
03d876d
to
a4cf48e
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 10 files changed, 2033 insertions(+), 1494 deletions(-)) |
Tests analyticsTotal tests: Click here to see the affected service packagesall service packages are affected Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccDataprocClusterIamPolicy |
Rerun these tests in REPLAYING mode to catch issues
|
name: annotations | ||
description: Allows clients to store small amounts of arbitrary data. | ||
- !ruby/object:Api::Type::String | ||
name: etag |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this being used by the DCL during requests? I'm seeing traits that indicate it may have been used
skip_test: true | ||
parameters: | ||
- !ruby/object:Api::Type::String | ||
name: name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this was annotated with the long_form trait. Can you add tests on the long form functionality?
a4cf48e
to
71b2e71
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 10 files changed, 2052 insertions(+), 1497 deletions(-)) |
Tests analyticsTotal tests: Click here to see the affected service packagesall service packages are affected Action takenFound 6 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccCloudbuildv2Connection_GheConnection|TestAccCloudbuildv2Connection_GithubConnection|TestAccCloudbuildv2Connection_GleConnection|TestAccCloudbuildv2Connection_GleOldConnection|TestAccCloudbuildv2Connection_GhePrivUpdateConnection|TestAccCloudbuildv2Connection_GlePrivUpdateConnection |
Rerun these tests in REPLAYING mode to catch issues
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change the changelog note to be similar to:
cloudbuildv2: changed underlying actuation engine for google_cloudbuildv2_repository, there should be no user-facing impact (hashicorp/terraform-provider-google#16969)
* Migrate cloudbuildv2 connection from DCL to mmv1 * Add conflicts and diff suppress func * Add long form name * Make etag a fingerprint
* Migrate cloudbuildv2 connection from DCL to mmv1 * Add conflicts and diff suppress func * Add long form name * Make etag a fingerprint
* Migrate cloudbuildv2 connection from DCL to mmv1 * Add conflicts and diff suppress func * Add long form name * Make etag a fingerprint
* Migrate cloudbuildv2 connection from DCL to mmv1 * Add conflicts and diff suppress func * Add long form name * Make etag a fingerprint
Migration of cloudbuild connection and repository.
fixes hashicorp/terraform-provider-google#16162
#9631 for repository only
Release Note Template for Downstream PRs (will be copied)